Skip to content

OIDC/sdk google recovery flow#69

Merged
matias-gonz merged 27 commits intofeat/oidc-account-recoveryfrom
oidc/sdk-google-recovery-flow
Feb 19, 2025
Merged

OIDC/sdk google recovery flow#69
matias-gonz merged 27 commits intofeat/oidc-account-recoveryfrom
oidc/sdk-google-recovery-flow

Conversation

@matias-gonz
Copy link
Member

@matias-gonz matias-gonz commented Feb 17, 2025

Description

Integrates OidcRecoveryValidator contract with SDK

@matias-gonz matias-gonz changed the base branch from feat/oidc-account-recovery to oidc/add-google-recovery-flow February 17, 2025 09:03
recovery: "0x4E619cA9DDb3A207E4764F3Ee5D36DD478212335",
recoveryOidc: "0x3ad654fC38bb5Abe789c912cfE900A867d52A164",
accountFactory: "0x01F99512191c036FcA9Fcd416dE73b19e93B7D60",
accountPaymaster: "0x2A2869fa5a1E94474798C02ED930278C154EE213",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we research if nuxt offers a better way to save this kind of data? It's kind of awful having to update this on every deploy. Maybe we can do something were the deploy script exports the addresses in some format that we can use as an input for this.

Not for this PR, though.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will add it to the backlog


return await client.addOidcAccount({
paymaster: {
address: paymasterAddress,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We have to ask about using the paymaster for these txs.

Base automatically changed from oidc/add-google-recovery-flow to feat/oidc-account-recovery February 19, 2025 11:53
@github-actions
Copy link

github-actions bot commented Feb 19, 2025

Visit the preview URL for this PR (updated for commit 749e76a):

https://zksync-auth-server-staging--pr69-oidc-sdk-google-reco-64lgfi8k.web.app

(expires Wed, 26 Feb 2025 11:57:26 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 509a9c9ea42583076f531c53cf2979c544d5d0b7

@matias-gonz matias-gonz merged commit bee7a38 into feat/oidc-account-recovery Feb 19, 2025
4 checks passed
@matias-gonz matias-gonz deleted the oidc/sdk-google-recovery-flow branch February 19, 2025 11:57
cpb8010 pushed a commit that referenced this pull request Jul 16, 2025
* Add Recover with Google button

* Add GoogleFlow

* Add useAddOidcAccount

* feat: add OidcValidator to account

* Add OidcRecoveryModuleAbi

* Add addOidcAccount to client

* Add addOidcAccount calldata

* Fix ABI

* fix: encodedOidcData

* feat: add oidcData parameter

* lint: sdk

* chore: lint oidc key registry server

* chore: lint auth-server

* chore: lint md

* chore: run prettier

* chore: add keywords to cspell

* chore: add keywords to cspell

* chore: update lock file

* chore: run prettier

* chore: lint auth-server

* chore: lint oidc key registry server

* fix: zeek nft

* fix: zeek nft
cpb8010 pushed a commit that referenced this pull request Jul 16, 2025
* Add Recover with Google button

* Add GoogleFlow

* Add useAddOidcAccount

* feat: add OidcValidator to account

* Add OidcRecoveryModuleAbi

* Add addOidcAccount to client

* Add addOidcAccount calldata

* Fix ABI

* fix: encodedOidcData

* feat: add oidcData parameter

* lint: sdk

* chore: lint oidc key registry server

* chore: lint auth-server

* chore: lint md

* chore: run prettier

* chore: add keywords to cspell

* chore: add keywords to cspell

* chore: update lock file

* chore: run prettier

* chore: lint auth-server

* chore: lint oidc key registry server

* fix: zeek nft

* fix: zeek nft
cpb8010 pushed a commit that referenced this pull request Jul 16, 2025
* Add Recover with Google button

* Add GoogleFlow

* Add useAddOidcAccount

* feat: add OidcValidator to account

* Add OidcRecoveryModuleAbi

* Add addOidcAccount to client

* Add addOidcAccount calldata

* Fix ABI

* fix: encodedOidcData

* feat: add oidcData parameter

* lint: sdk

* chore: lint oidc key registry server

* chore: lint auth-server

* chore: lint md

* chore: run prettier

* chore: add keywords to cspell

* chore: add keywords to cspell

* chore: update lock file

* chore: run prettier

* chore: lint auth-server

* chore: lint oidc key registry server

* fix: zeek nft

* fix: zeek nft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants